Skip to content

RL: Use string examples instead of int#256

Merged
connermanuel merged 1 commit intomainfrom
conner/seed-examples
Apr 28, 2026
Merged

RL: Use string examples instead of int#256
connermanuel merged 1 commit intomainfrom
conner/seed-examples

Conversation

@connermanuel
Copy link
Copy Markdown
Contributor

We have fields that accept oneOf: string, int. Our examples are int. This changes it to string. Why? Because stainless runs tests on these examples, and when the type is oneOf it will only validate the example against the first type. So it's comparing the ints to the type string and failing all the checks.

Yes, the other solution is to change oneOf: string, int to oneOf: int, string.

@github-actions
Copy link
Copy Markdown

github-actions Bot commented Apr 27, 2026

✱ Stainless preview builds

This PR will update the togetherai SDKs with the following commit messages.

go

chore(internal): regenerate SDK with no functional changes

openapi

chore(internal): regenerate SDK with no functional changes

python

chore(internal): regenerate SDK with no functional changes

terraform

chore(internal): regenerate SDK with no functional changes

typescript

chore(internal): regenerate SDK with no functional changes
togetherai-openapi studio · code

Your SDK build had at least one "note" diagnostic.
generate ✅

⚠️ togetherai-python studio · code

Your SDK build had at least one "warning" diagnostic.
generate ⚠️build ⏭️lint ⏭️test ⏭️

⚠️ togetherai-typescript studio · conflict

Your SDK build had at least one warning diagnostic.

⚠️ togetherai-go studio · code

Your SDK build had a failure in the test CI job, which is a regression from the base state.
generate ✅build ⏭️lint ✅test ❗

go get github.com/stainless-sdks/togetherai-go@966018dbe34b1404da4017ed84390b0f92e526e2
togetherai-terraform studio · code

Your SDK build had at least one "note" diagnostic.
generate ✅lint ✅test ✅


This comment is auto-generated by GitHub Actions and is automatically kept up to date as you push.
If you push custom code to the preview branch, re-run this workflow to update the comment.
Last updated: 2026-04-28 00:49:29 UTC

Copy link
Copy Markdown

@felix-588 felix-588 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Looks good to me

@connermanuel connermanuel merged commit 0cc4429 into main Apr 28, 2026
6 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants